Skip to content

Conversation

@huntergregory
Copy link
Contributor

@huntergregory huntergregory commented Jan 7, 2025

Reason for Change:

Include counts for CIDR and endport policies. New NPM heartbeat sent to app insights:

info: NPM heartbeat. Total policies: 50, CIDR policies: 10, EndPort policies: 3

Issue Fixed:

Requirements:

Notes:

@huntergregory huntergregory added enhancement npm Related to NPM. labels Jan 7, 2025
@huntergregory huntergregory requested a review from a team as a code owner January 7, 2025 02:26
@huntergregory
Copy link
Contributor Author

/azp run Azure Container Networking PR

@huntergregory
Copy link
Contributor Author

/azp run NPM Conformance Tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

1 similar comment
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@huntergregory
Copy link
Contributor Author

/azp run Azure Container Networking PR, NPM Conformance Tests, NPM Scale Test

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

Signed-off-by: Hunter Gregory <[email protected]>
@huntergregory
Copy link
Contributor Author

/azp run Azure Container Networking PR, NPM Conformance Tests, NPM Scale Test

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@huntergregory
Copy link
Contributor Author

windows conformance had an infra failure. ok to ignore

@huntergregory huntergregory changed the title log: [NPM] heartbeat includes number of cidr and named port netpols log: [NPM] heartbeat includes number of cidr and endport netpols Jan 9, 2025
@huntergregory
Copy link
Contributor Author

/azp run Azure Container Networking PR, NPM Conformance Tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@huntergregory huntergregory force-pushed the huntergregory/netpol-count branch from e8a2832 to b400224 Compare January 9, 2025 22:08
@huntergregory
Copy link
Contributor Author

/azp run Azure Container Networking PR, NPM Conformance Tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@huntergregory huntergregory force-pushed the huntergregory/netpol-count branch from b400224 to 49771e3 Compare January 9, 2025 22:45
@huntergregory
Copy link
Contributor Author

/azp run Azure Container Networking PR, NPM Conformance Tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@huntergregory
Copy link
Contributor Author

conformance failure was a flake:

[FAILED] Unable to create netpol netpol-x-1471/allow-all: unable to create network policy netpol-x-1471/allow-all: Post "[https://conformanc-e2e-npm-2025-01--d9eabe-6i2hcs7r.hcp.westus2.azmk8s.io:443/apis/networking.k8s.io/v1/namespaces/netpol-x-1471/networkpolicies"](https://conformanc-e2e-npm-2025-01--d9eabe-6i2hcs7r.hcp.westus2.azmk8s.io/apis/networking.k8s.io/v1/namespaces/netpol-x-1471/networkpolicies%22): read tcp 10.1.18.15:46760->52.148.147.20:443: read: connection reset by peer

@huntergregory
Copy link
Contributor Author

Don't need this change any more

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement npm Related to NPM.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants